[App Size reduction] Remove dark variation duplicate of blaze intro asset #15763
+0
−10
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
iamgabrielma
added
type: enhancement
Collaborator
|
|
RafaelKayumov
approved these changes
Jun 17, 2025
Contributor
RafaelKayumov
left a comment
RafaelKayumov
left a comment
There was a problem hiding this comment.
The additional PDF seems to be added in Update dark mode image for blaze intro illustration as a part of Blaze: Update dark mode color for intro view and campaign creation form
Both PDFs look identical to me and same for AI:
🧾 PDF Comparison Summary
Compared the following two PDF assets to evaluate redundancy:
blaze-intro-illustration.pdfillustraion.pdf
✅ Result: Files are Identical
- No differences found in content, layout, vector graphics, or embedded text.
- Same number of pages and objects.
- Nearly identical file sizes.
- Suitable for both light and dark themes without visual conflict.
🧹 Recommendation
One of the files can be safely removed from the app bundle to reduce size without affecting functionality or visual appearance.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR removes
illustraion.pdffrom the asset library, removing 3.1MB from the pre-thinning build as part of reducing app size on WOOMOB-534 and using the same image to render as universal in both variants.I'm not quite sure what this asset is doing, as is exactly the same as its light variant
blaze-intro-illustration.pdf. Both also have the same transparency, so there's seem to be no palpable differences between assets.I'm not sure either how to trigger this onboarding without hardcoding the view somewhere in the app, so I'm adding Kiwi as reviewers. Please double check we don't crash due a missing asset, since we tend to use force unwrap around image assets.
Testing information
Blaze intro view should display as always, both light and dark modes.